Skip to content

Add support for OpenAI's gpt-5.2-codex #2166

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
naorpeled merged 1 commit into from
Jan 17, 2026
Merged

Add support for OpenAI's gpt-5.2-codex #2166

naorpeled merged 1 commit into from
Jan 17, 2026

Conversation

@PeterDaveHello
Copy link
Contributor

@PeterDaveHello PeterDaveHello commented Jan 14, 2026 •
edited by qodo-free-for-open-source-projects bot
Loading

User description

Reference:

PR Type

Enhancement

Description

  • Add gpt-5.2-codex model with 400K token context window

  • Mark gpt-5.2-codex as unsupported for temperature parameter

Diagram Walkthrough

flowchart LR
  A["gpt-5.2-codex Model"] -- "400K context tokens" --> B["Model Configuration"]
  A -- "no temperature support" --> C["NO_SUPPORT_TEMPERATURE_MODELS"]
Loading

File Walkthrough
Relevant files
Enhancement
__init__.py
Register gpt-5.2-codex model configuration                             

pr_agent/algo/init.py

  • Added gpt-5.2-codex entry to model context window mapping with 400K
    tokens
  • Added gpt-5.2-codex to NO_SUPPORT_TEMPERATURE_MODELS list to disable
    temperature parameter support
 +2/-0     

@qodo-free-for-open-source-projects
Copy link
Contributor

qodo-free-for-open-source-projects bot commented Jan 14, 2026

PR Compliance Guide 🔍

Below is a summary of compliance checks for this PR:

Security Compliance
🟢
No security concerns identified No security vulnerabilities detected by AI analysis. Human verification advised for critical code.
Ticket Compliance
⚪
🎫 No ticket provided
  • Create ticket/issue
Codebase Duplication Compliance
⚪
Codebase context is not defined

Follow the guide to enable codebase context checks.

Custom Compliance
🟢
Consistent Naming Conventions

Objective: All new variables, functions, and classes must follow the project's established naming
standards

Status: Passed

No Dead or Commented-Out Code

Objective: Keep the codebase clean by ensuring all submitted code is active and necessary

Status: Passed

Robust Error Handling

Objective: Ensure potential errors and edge cases are anticipated and handled gracefully throughout
the code

Status: Passed

Single Responsibility for Functions

Objective: Each function should have a single, well-defined responsibility

Status: Passed

When relevant, utilize early return

Objective: In a code snippet containing multiple logic conditions (such as 'if-else'), prefer an
early return on edge cases than deep nesting

Status: Passed
Compliance status legend 🟢 - Fully Compliant
🟡 - Partial Compliant
🔴 - Not Compliant
⚪ - Requires Further Human Verification
🏷️ - Compliance label

@qodo-free-for-open-source-projects
Copy link
Contributor

qodo-free-for-open-source-projects bot commented Jan 14, 2026

PR Code Suggestions ✨

Explore these optional code suggestions:

CategorySuggestion                                                                                                                                    Impact
Possible issue
Verify model token limit is correct

Verify the token limit for gpt-5.2-codex against official documentation, as it
differs from another variant, gpt-5.2-chat-latest.

pr_agent/algo/init.py [43]

-'gpt-5.2-codex': 400000,  # 400K, but may be limited by config.max_model_tokens
+'gpt-5.2-codex': 400000,  # 400K, but may be limited by config.max_model_tokens. TODO: Verify token limit from official documentation.
  • Apply / Chat
Suggestion importance[1-10]: 5

__

Why: The suggestion correctly points out a potential inconsistency in token limits among related models and reasonably asks for verification, which is a good practice for maintaining correctness.

Low
  • More
  • Author self-review: I have reviewed the PR code suggestions, and addressed the relevant ones.

@naorpeled
Copy link
Collaborator

naorpeled commented Jan 17, 2026

LGTM!
Thanks for this 🙏

@naorpeled naorpeled merged commit f654143 into qodo-ai:main Jan 17, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@naorpeled naorpeled naorpeled approved these changes

Assignees

No one assigned

Labels

Review effort 1/5

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@PeterDaveHello @naorpeled